Skip to content

Update stacks plugin binary download to accommodate Zip contents in CRT style release#37211

Merged
Maed223 merged 2 commits intomainfrom
update-stacks-plugin-binary-download-to-accomodate-crt-zip
Jun 5, 2025
Merged

Update stacks plugin binary download to accommodate Zip contents in CRT style release#37211
Maed223 merged 2 commits intomainfrom
update-stacks-plugin-binary-download-to-accomodate-crt-zip

Conversation

@Maed223
Copy link
Copy Markdown
Contributor

@Maed223 Maed223 commented Jun 4, 2025

Fixes: #37192

Description

There was an overlooked detail in the binary manager for plugins, in that we didn't properly accommodate the CRT style of releases. In the CRT release process there is a LICENSE.txt that we didn't account for, leading to the decompress failing because the file limit was set too low. Additionally there is some file path adjustments made to properly access the binary.

For testing, using a local build of this branch, you should be able to run terraform stacks

Target Release

1.13.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@Maed223 Maed223 force-pushed the update-stacks-plugin-binary-download-to-accomodate-crt-zip branch from 6dad9d3 to bab03c3 Compare June 4, 2025 18:24
@Maed223 Maed223 added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Jun 4, 2025
@Maed223 Maed223 force-pushed the update-stacks-plugin-binary-download-to-accomodate-crt-zip branch from bab03c3 to e16c5f6 Compare June 4, 2025 20:24
@Maed223 Maed223 marked this pull request as ready for review June 4, 2025 20:34
@Maed223 Maed223 requested a review from a team as a code owner June 4, 2025 20:34
@Maed223 Maed223 force-pushed the update-stacks-plugin-binary-download-to-accomodate-crt-zip branch from 9bd9d76 to e16c5f6 Compare June 4, 2025 21:10
@Maed223 Maed223 merged commit e9f6f2e into main Jun 5, 2025
12 checks passed
@Maed223 Maed223 deleted the update-stacks-plugin-binary-download-to-accomodate-crt-zip branch June 5, 2025 23:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 6, 2025

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot use terraform stacks

2 participants